-
Notifications
You must be signed in to change notification settings - Fork 5.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Context Window Exceeded fix #4977
Conversation
00597d6
to
d85debd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me, thank you for doing it!
It'd be nice if we could do an eval before merging just to make sure that things are working, I added the label now.
Hi! I started running the evaluation on your PR. You will receive a comment with the results shortly. |
Unfortunately it looks like the Deepseek eval workflow doesn't work on But I can submit it to a number of good tests tomorrow! |
I think we don't need to test then, this should be fine! |
b6ec632
to
0b7c08d
Compare
0b7c08d
to
2fcb088
Compare
End-user friendly description of the problem this fixes or functionality that this introduces
Truncate agent history when context window is exceeded
Give a summary of what the PR does, explaining any non-trivial design decisions
This PR proposes a simple fix for the ContextWindowExceededError:
Please note that this is not well tested, but it is ready for code review.Link of any specific issues this addresses
Fix #4951
Fix #4894
To run this PR locally, use the following command: